-
Notifications
You must be signed in to change notification settings - Fork 652
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat:: Added --insecure
of CLI argument
#2696
Conversation
|
...elemetry-exporter-jaeger-proto-grpc/src/opentelemetry/exporter/jaeger/proto/grpc/__init__.py
Outdated
Show resolved
Hide resolved
...elemetry-exporter-jaeger-proto-grpc/src/opentelemetry/exporter/jaeger/proto/grpc/__init__.py
Outdated
Show resolved
Hide resolved
...elemetry-exporter-jaeger-proto-grpc/src/opentelemetry/exporter/jaeger/proto/grpc/__init__.py
Show resolved
Hide resolved
...elemetry-exporter-jaeger-proto-grpc/src/opentelemetry/exporter/jaeger/proto/grpc/__init__.py
Outdated
Show resolved
Hide resolved
...elemetry-exporter-jaeger-proto-grpc/src/opentelemetry/exporter/jaeger/proto/grpc/__init__.py
Outdated
Show resolved
Hide resolved
@tonycody we can get this merged if you can address the minor comment and fix the failures. |
@tonycody ping. |
@srikanthccv ping. |
@tonycody Add changelog and fix lint. Also some new commits are added with |
@tonycody your last four commits didn't have user id and preventing from easycla check to pass #2696 (comment). |
This reverts commit ae43bfb.
@srikanthccv ping |
/easycla |
@lzchen @srikanthccv review |
@tonycody fix the lint. |
@srikanthccv @lzchen ping. |
Description
CLI added the parameter '--insecure' to support whether or not to communicate securely. The opentElemetry auto instrument cannot use the 'insecure' parameter
Fixes # (issue)